-
-
Notifications
You must be signed in to change notification settings - Fork 401
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Parametrize remote fixtures #2255
base: main
Are you sure you want to change the base?
Conversation
Hello @volodymyrss! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:
Comment last updated at 2022-03-26 08:59:26 UTC |
@volodymyrss - Is this still relevant? If yes, could you remove the commits that are already in |
80b2ec1
to
e127104
Compare
It's still relevant, it's meant to relocate the fixtures for this test parametrization (creating remote data automatically) to common location, not specific to heasarc. |
That sounds all good, as you may have seen we don't put a time limit on PRs in this repo. However, I removed the milestone so it won't pop up at release time. |
No description provided.